Skip to content
This repository was archived by the owner on Dec 21, 2023. It is now read-only.

Replace bootstrap <button> with identically styled <a> hyperlinks#7

Merged
InfinityGhost merged 1 commit intoOpenTabletDriver:masterfrom
vgf89:master
Aug 5, 2021
Merged

Replace bootstrap <button> with identically styled <a> hyperlinks#7
InfinityGhost merged 1 commit intoOpenTabletDriver:masterfrom
vgf89:master

Conversation

@vgf89
Copy link
Copy Markdown
Contributor

@vgf89 vgf89 commented Aug 4, 2021

Makes the buttons act like regular links, using the same bootstrap style stuff as the bootstrap buttons. The URL can be previewed, middle-clicking with mouse opens in new tab, etc etc.

@vgf89
Copy link
Copy Markdown
Contributor Author

vgf89 commented Aug 4, 2021

I haven't actually tested this with my own build since I can't get my Visual Studio instance to see .net5, but it's a trivial change that should be easy to test. I tried making the same change in the browser on the generated html at least though and it did work.

EDIT: I'm updating Visual Studio, .net5 should work once it's updated. Testing incoming hopefully.

@vgf89
Copy link
Copy Markdown
Contributor Author

vgf89 commented Aug 5, 2021

Just got it running on my VS. All the buttons work like normal links and look the same as before. The fade animation still works. It is ready to merge whenever.

@vgf89 vgf89 changed the title Replace bootstrap buttons with similarly styled hyperlinks Replace bootstrap <button>s with similarly styled <a> hyperlinks Aug 5, 2021
@vgf89 vgf89 changed the title Replace bootstrap <button>s with similarly styled <a> hyperlinks Replace bootstrap <button> with identically styled <a> hyperlinks Aug 5, 2021
@InfinityGhost InfinityGhost added the enhancement New feature or request label Aug 5, 2021
@InfinityGhost InfinityGhost merged commit d75ff43 into OpenTabletDriver:master Aug 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants